Skip to content

fix: test start-screen for --dry-run mode for solve and hive commands (issue #539)#1430

Open
konard wants to merge 15 commits into
mainfrom
issue-539-ff6d7f33ccd8
Open

fix: test start-screen for --dry-run mode for solve and hive commands (issue #539)#1430
konard wants to merge 15 commits into
mainfrom
issue-539-ff6d7f33ccd8

Conversation

@konard

@konard konard commented Mar 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes #539

Problem

start-screen hive "https://github.com/konard" --dry-run --once --verbose was failing with "Invalid GitHub URL: missing owner/repo" because start-screen.mjs was not using the same URL validation logic as the hive and solve commands.

Root Cause

The original start-screen.mjs had its own URL validation that required both owner and repo, rejecting valid user/organization URLs like https://github.com/konard.

Fix

Accepted URL types

  • ✅ User URLs: https://github.com/owner
  • ✅ Repository URLs: https://github.com/owner/repo
  • ✅ Issue URLs: https://github.com/owner/repo/issues/123
  • ✅ Pull Request URLs: https://github.com/owner/repo/pull/456

Tests

All tests pass (included in CI):

Testing --dry-run mode for solve command...
  solve --dry-run: ✓ PASSED

Testing --dry-run mode for hive command with user URL...
  hive --dry-run user URL: ✓ PASSED

Testing --dry-run mode for hive command with repo URL...
  hive --dry-run repo URL: ✓ PASSED

All tests PASSED! ✓

🤖 Generated with Claude Code

@konard konard self-assigned this Mar 14, 2026
@konard

konard commented Mar 30, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-03-30T18:39:52.994Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard konard changed the title [WIP] We need to make sure we test start-screen for --dry-run mode for both solve and hive commands fix: test start-screen for --dry-run mode for solve and hive commands (issue #539) Mar 30, 2026
@konard
konard marked this pull request as ready for review March 30, 2026 18:46
@konard

konard commented Mar 30, 2026

Copy link
Copy Markdown
Contributor Author

✅ Ready to merge

This pull request is now ready to be merged:

  • All CI checks have passed
  • No merge conflicts
  • No pending changes

Monitored by hive-mind with --auto-restart-until-mergeable flag

@konard
konard marked this pull request as draft April 11, 2026 04:35
@konard

konard commented Apr 11, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-04-11T04:35:03.959Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard
konard marked this pull request as ready for review April 11, 2026 04:37
@konard

konard commented Apr 11, 2026

Copy link
Copy Markdown
Contributor Author

✅ Ready to merge

All requirements for issue #539 are satisfied:

  • Fix: src/start-screen.mjs uses the shared parseGitHubUrl from github.lib.mjs, accepting user/org URLs like https://github.com/konard (consistent with hive and solve commands)
  • Tests: tests/test-start-screen.mjs includes --dry-run tests for both solve and hive commands (including the exact failing case from the issue)
  • CI: .github/workflows/release.yml runs test-start-screen.mjs as a check step

All CI checks pass. PR has no merge conflicts.


Reviewed by AI work session

@konard

konard commented Apr 26, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-04-26T17:53:51.485Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard
konard marked this pull request as draft April 26, 2026 17:53
@konard
konard marked this pull request as ready for review April 26, 2026 17:59
@konard

konard commented Apr 26, 2026

Copy link
Copy Markdown
Contributor Author

✅ Ready to merge

All requirements for issue #539 are satisfied:

  • Fix: src/start-screen.mjs uses the shared parseGitHubUrl from github.lib.mjs, accepting user/org URLs like https://github.com/konard (consistent with hive and solve commands)
  • Tests: tests/test-start-screen.mjs includes --dry-run tests for both solve and hive commands (including the exact failing case from the issue)
  • CI: .github/workflows/release.yml runs test-start-screen.mjs as a check step

Branch has been synced with latest main and all tests pass locally. CI is running.


Reviewed by AI work session

@konard

konard commented Apr 26, 2026

Copy link
Copy Markdown
Contributor Author

🔄 Auto-restart triggered (iteration 1)

Reason: CI failures detected

Starting new session to address the issues.


Auto-restart-until-mergeable mode is active. This run will stop after 5 restart iterations.

@konard
konard marked this pull request as draft April 26, 2026 18:10
@konard

konard commented Apr 26, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-04-26T18:10:49.643Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard
konard marked this pull request as ready for review April 26, 2026 18:11
@konard

konard commented Apr 26, 2026

Copy link
Copy Markdown
Contributor Author

✅ Ready to merge

All requirements for issue #539 are satisfied:

  • Fix: src/start-screen.mjs uses the shared parseGitHubUrl from github.lib.mjs, accepting user/org URLs like https://github.com/konard (consistent with hive and solve commands)
  • Tests: tests/test-start-screen.mjs includes --dry-run tests for both solve and hive commands (including the exact failing case from the issue)
  • CI: .github/workflows/release.yml runs test-start-screen.mjs as a check step
  • Changeset: Added for version tracking

Latest CI run (run #24963479278): ✅ success
PR status: mergeable, no conflicts.


Reviewed by AI work session

@konard
konard marked this pull request as draft April 26, 2026 21:19
@konard

konard commented Apr 26, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-04-26T21:19:36.611Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard
konard marked this pull request as ready for review April 26, 2026 21:22
@konard

konard commented Apr 26, 2026

Copy link
Copy Markdown
Contributor Author

✅ Ready to merge

All requirements for issue #539 are satisfied:

  • Fix: src/start-screen.mjs uses the shared parseGitHubUrl from github.lib.mjs, accepting user/org URLs like https://github.com/konard (consistent with hive and solve commands)
  • Tests: tests/test-start-screen.mjs includes --dry-run tests for both solve and hive commands (including the exact failing case from the issue)
  • CI: .github/workflows/release.yml runs test-start-screen.mjs as a check step
  • Changeset: Added for version tracking

All 12 tests pass locally. CI run #24963479278: success. PR is mergeable with no conflicts.


Reviewed by AI work session

@konard

konard commented Apr 26, 2026

Copy link
Copy Markdown
Contributor Author

Solution summary

PR 1430 is ready to merge. Here's the current state:

The fix was already complete from the previous sessions — src/start-screen.mjs uses the shared parseGitHubUrl function, and tests for both solve and hive --dry-run modes are in CI. Nothing new needed to be changed; I just verified the state and marked the PR ready.


This summary was automatically extracted from the AI working session output.

@konard
konard marked this pull request as draft April 28, 2026 13:35
@konard

konard commented Apr 28, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-04-28T13:35:51.657Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard
konard marked this pull request as ready for review April 28, 2026 13:38
@konard

konard commented Apr 28, 2026

Copy link
Copy Markdown
Contributor Author

✅ Ready to merge

All requirements for issue #539 are satisfied:

  • Fix: src/start-screen.mjs uses the shared parseGitHubUrl from github.lib.mjs, accepting user/org URLs like https://github.com/konard (consistent with hive and solve commands)
  • Tests: tests/test-start-screen.mjs includes --dry-run tests for both solve and hive commands (including the exact failing case from the issue)
  • CI: .github/workflows/release.yml runs test-start-screen.mjs as a check step
  • Changeset: Added for version tracking

Branch synced with latest main (includes issue #1703 fix). All 12 local tests pass. CI running on the new push.


Reviewed by AI work session

@konard

konard commented Apr 28, 2026

Copy link
Copy Markdown
Contributor Author

Solution summary

The PR is in good shape. Here's the summary:

CI is now running on the new push. The PR is at #1430.


This summary was automatically extracted from the AI working session output.

@konard
konard marked this pull request as draft April 28, 2026 14:18
@konard

konard commented Apr 28, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-04-28T14:18:16.758Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-05-06T11:49:30.329Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

Done. Here's what was done this session:

  1. Reviewed PR state: PR was in draft mode after the last AI work session start.
  2. Checked branch vs main: Branch issue-539-ff6d7f33ccd8 is already up to date with main (v1.67.0) — no new commits to merge.
  3. CI status: All 5 recent CI runs passed ✅ (latest on 86303b47).
  4. Marked PR fix: test start-screen for --dry-run mode for solve and hive commands (issue #539) #1430 as ready for review — it is now OPEN, not draft, MERGEABLE, CLEAN.

The PR diff from main remains only .changeset/fix-start-screen-url-validation-539.md — the fix and tests were already merged to main in earlier sessions.


This summary was automatically extracted from the AI working session output.

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

Done. Here's what was done this session:

  1. Reviewed PR state: PR fix: test start-screen for --dry-run mode for solve and hive commands (issue #539) #1430 was in draft mode (set by the "AI Work Session Started" marker at the start of this session).
  2. Checked branch vs main: Branch issue-539-ff6d7f33ccd8 is already up to date with main (v1.67.0) — no new commits to merge.
  3. CI status: All 5 recent CI runs passed ✅ (latest on 86303b47).
  4. Marked PR as ready for review — it is now OPEN, MERGEABLE, CLEAN.

The PR diff from main remains only .changeset/fix-start-screen-url-validation-539.md. The actual fix and tests were already merged to main in earlier sessions. PR: #1430


This summary was automatically extracted from the AI working session output.

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-05-06T12:10:14.778Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

PR #1430 is now ready for review. Here's what was done this session:

  1. Reviewed PR state: PR was in draft mode after the last AI work session start.
  2. Checked branch vs main: Branch issue-539-ff6d7f33ccd8 is already up to date with main (v1.67.0) — no new commits needed.
  3. CI status: All 5 recent CI runs passed ✅ (latest: 86303b47, run 25432583400)
  4. Marked PR as ready for review (was in draft mode).

Current state:

  • State: OPEN, not draft, MERGEABLE
  • Merge state: CLEAN — no conflicts
  • PR diff from main: Only .changeset/fix-start-screen-url-validation-539.md (the actual fix src/start-screen.mjs using parseGitHubUrl and the --dry-run tests were already merged to main in earlier sessions)

PR: #1430

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

PR #1430 is now ready for review. Here's the current state:

  • Branch: Up to date with main (v1.67.0), no new commits needed
  • CI: All 5 recent runs passed ✅
  • Merge state: CLEAN, MERGEABLE, no conflicts
  • Diff from main: Only .changeset/fix-start-screen-url-validation-539.md — the actual fix (src/start-screen.mjs using parseGitHubUrl) and --dry-run tests were already merged to main in previous sessions

#1430


This summary was automatically extracted from the AI working session output.

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-05-06T12:35:24.469Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

Done. Here's what was done this session:

  1. Reviewed PR state: PR fix: test start-screen for --dry-run mode for solve and hive commands (issue #539) #1430 was in draft mode (set by the AI work session start marker).
  2. Checked branch vs main: Branch issue-539-ff6d7f33ccd8 is already up to date with main (v1.67.0) — no new commits needed.
  3. CI status: All 5 recent CI runs passed ✅ (latest: 86303b47, run 25432583400).
  4. Marked PR as ready for review — it is now OPEN, not draft, MERGEABLE, CLEAN.

Current state:

  • Diff from main: Only .changeset/fix-start-screen-url-validation-539.md — the actual fix (src/start-screen.mjs using parseGitHubUrl) and --dry-run tests were already merged to main in earlier sessions.

#1430

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

PR #1430 is now ready for review. Here's the current state:

  • Branch: Up to date with main (v1.67.0), no new commits needed
  • CI: All 5 recent runs passed ✅
  • Merge state: CLEAN, MERGEABLE, no conflicts
  • Diff from main: Only .changeset/fix-start-screen-url-validation-539.md — the actual fix (src/start-screen.mjs using parseGitHubUrl) and --dry-run tests were already merged to main in previous sessions

This summary was automatically extracted from the AI working session output.

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-05-06T12:55:47.753Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

PR #1430 is now ready for review. Here's the current state:

  • Branch: Up to date with main (v1.67.0), no new commits needed
  • CI: All 5 recent runs passed ✅ (latest on commit 86303b47, success)
  • Merge state: CLEAN, MERGEABLE, no conflicts
  • Diff from main: Only .changeset/fix-start-screen-url-validation-539.md — the actual fix (src/start-screen.mjs using parseGitHubUrl) and --dry-run tests were already merged to main in previous sessions

#1430

@konard

konard commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

PR #1430 is now marked as ready for review. Current state:

  • State: OPEN, not draft, MERGEABLE, CLEAN
  • CI: All 5 recent runs passed ✅ (latest on commit 86303b47)
  • Diff from main: Only .changeset/fix-start-screen-url-validation-539.md — the actual code fix and tests were merged to main in earlier sessions
  • No uncommitted changes, working tree is clean

#1430


This summary was automatically extracted from the AI working session output.

@konard

konard commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-05-07T10:53:44.877Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard

konard commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

Done. Here's what was done this session:

  1. Merged latest main (v1.67.2) into the branch — main had advanced with case studies for issues 1761 and 1763, plus fixes for solve.auto-merge, solve.watch, and additional tests since the last merge.
  2. Pushed the updated branch to remote (1ba28fff).
  3. Marked PR fix: test start-screen for --dry-run mode for solve and hive commands (issue #539) #1430 as ready for review — it is OPEN, MERGEABLE.

CI is queued on the new merge commit. All 5 previous CI runs passed ✅.

The PR diff from main remains only .changeset/fix-start-screen-url-validation-539.md — the actual fix and tests were already merged to main in earlier sessions.

#1430


This summary was automatically extracted from the AI working session output.

@konard

konard commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

Done. Here's what was accomplished this session:

  1. Merged latest main (v1.67.2) — main had advanced with case studies for issues 1761 and 1763, plus fixes to solve.auto-merge, solve.watch, and additional tests.
  2. Pushed the updated branch (1ba28fff).
  3. Marked PR fix: test start-screen for --dry-run mode for solve and hive commands (issue #539) #1430 as ready for review — OPEN, MERGEABLE.

CI is now queued on the new merge commit. The PR diff from main remains only .changeset/fix-start-screen-url-validation-539.md — the actual code fix and tests were already merged to main in earlier sessions.

PR: #1430


This summary was automatically extracted from the AI working session output.

@konard

konard commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-05-07T15:42:51.543Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard

konard commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

PR #1430 is now ready for review.

What was done this session:

  1. Merged latest main (v1.68.0) into the branch — main had advanced with folder-based test discovery and start-screen deprecation (issue We should remove from tests tasks (both local tests and CI/CD tests), that don't execute in dry mode, also all tests should switch to --isolated screen modes, as we don't use regular start-screen commands anymore #1758) since the last merge.
  2. Pushed the updated branch to remote (abe8cd92).
  3. Marked PR as ready for review — it is now OPEN, not draft, MERGEABLE.

PR diff from main: Only .changeset/fix-start-screen-url-validation-539.md — the actual fix (src/start-screen.mjs using parseGitHubUrl from github.lib.mjs) and --dry-run tests (tests/test-start-screen.mjs) were already merged to main in previous sessions. This PR carries only the changeset file for the release notes.

CI is queued on the new merge commit. All 5 previous CI runs passed ✅.


This summary was automatically extracted from the AI working session output.

@konard

konard commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

Working session summary

PR #1430 is now ready for review. Here's what was done this session:

  1. Merged latest main (v1.68.0) into the branch — main had advanced with folder-based test discovery and start-screen deprecation changes (issue We should remove from tests tasks (both local tests and CI/CD tests), that don't execute in dry mode, also all tests should switch to --isolated screen modes, as we don't use regular start-screen commands anymore #1758) since the last merge.
  2. Pushed the updated branch to remote (abe8cd92).
  3. Marked PR fix: test start-screen for --dry-run mode for solve and hive commands (issue #539) #1430 as ready for review — it is OPEN, MERGEABLE, not draft.

PR diff from main: Only .changeset/fix-start-screen-url-validation-539.md — the actual fix (src/start-screen.mjs using parseGitHubUrl) and --dry-run tests were already merged to main in previous sessions. This PR carries only the changeset file for the release notes.

CI is queued on the new merge commit; all 5 previous runs passed ✅.

PR: #1430


This summary was automatically extracted from the AI working session output.

@konard

konard commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-05-07T19:48:33.609Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard

konard commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

PR #1430 is now ready for review. Here's the current state:

  • Merged latest main (v1.69.0) — main had advanced with the --show-limits Telegram feature and related modules.
  • Pushed the updated branch (commit 539a8ee1).
  • Marked PR as ready for review — OPEN, not draft, MERGEABLE.
  • Merge state: UNSTABLE (CI is queued on the new merge commit, which is expected).
  • PR diff from main: Only .changeset/fix-start-screen-url-validation-539.md — the actual fix (src/start-screen.mjs using parseGitHubUrl) and --dry-run tests were already merged to main in earlier sessions.

All 5 previous CI runs passed ✅. PR: #1430


This summary was automatically extracted from the AI working session output.

@konard

konard commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-05-07T20:12:51.802Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard

konard commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

PR #1430 is now ready for review. Here's the current state:

  • State: OPEN, not draft, MERGEABLE
  • Branch diff from main: Only .changeset/fix-start-screen-url-validation-539.md — the actual fix (src/start-screen.mjs using parseGitHubUrl from github.lib.mjs) and --dry-run tests (tests/test-start-screen.mjs) were already merged to main in previous sessions
  • CI: All 5 recent runs passed ✅ (latest on commit 539a8ee1)
  • No conflicts, no uncommitted changes

The PR is at #1430 and awaiting merge.


This summary was automatically extracted from the AI working session output.

@konard

konard commented May 8, 2026

Copy link
Copy Markdown
Contributor Author

🤖 AI Work Session Started

Starting automated work session at 2026-05-08T18:51:13.686Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard

konard commented May 8, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

PR #1430 is now ready for review.

What was done this session:

  1. Merged latest main into the branch — main had advanced with 5 commits since the last merge (issue-1758 follow-up: unsafe tests exclusion, case study refresh, log upload test fixes, start-screen test refactor).
  2. Pushed the updated branch to remote (b05d9985).
  3. Marked PR as ready for review — OPEN, not draft, MERGEABLE.

PR diff from main: Only .changeset/fix-start-screen-url-validation-539.md — the actual fix (src/start-screen.mjs using parseGitHubUrl from github.lib.mjs) and --dry-run tests (tests/test-start-screen.mjs) were already merged to main in previous sessions.

CI: Queued on the new merge commit. All 5 previous CI runs passed ✅.

@konard

konard commented May 8, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

PR #1430 is updated and ready for review. Here's what was done:

  1. Merged latest main — 5 new commits (issue-1758 follow-up fixes: unsafe test exclusion, log upload test refactor, start-screen test refactor, case study refresh).
  2. Pushed the updated branch (b05d9985).
  3. Marked PR as ready for review — OPEN, MERGEABLE, not draft.

The PR diff from main remains only the changeset file .changeset/fix-start-screen-url-validation-539.md. The actual fix and tests were already merged to main in earlier sessions. CI is queued on the new commit.


This summary was automatically extracted from the AI working session output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

We need to make sure we test start-screen for --dry-run mode for both solve and hive commands

1 participant